Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TeX file adjustments. #971

Merged
merged 3 commits into from
May 28, 2024
Merged

TeX file adjustments. #971

merged 3 commits into from
May 28, 2024

Conversation

sudorook
Copy link
Contributor

@sudorook sudorook commented May 3, 2024

  1. Interpret .cls files as LaTeX.
  2. Some TeX editors and other programs use a % !TEX and % !BIB macro at the top of .tex and .bib files, respectively, so I added those as 'shebangs' in the comment style classes. That way, the comment block is injected after the macros.

Copy link
Member

@carmenbianca carmenbianca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good to me. Thanks @sudorook !

Can you add an entry in the change log?

@sudorook
Copy link
Contributor Author

sudorook commented May 3, 2024

Do I just commit and push changes to CHANGELOG.md?

@carmenbianca
Copy link
Member

Yes, a simple edit to CHANGELOG.md, but I think you added a whole other heap of commits as well 😅

@sudorook
Copy link
Contributor Author

sudorook commented May 3, 2024

Oh, I'm just seeing that. Sorry... I've been rebasing branches locally and didn't realize that happened.

@sudorook
Copy link
Contributor Author

sudorook commented May 3, 2024

Okay, should be fine now. I'm still not sure how it happened, but as long as everything works...

@carmenbianca
Copy link
Member

Fantastic, thank you! I will merge soon.

@carmenbianca carmenbianca merged commit 4a17cf0 into fsfe:main May 28, 2024
14 checks passed
@sudorook sudorook deleted the tex_fixes branch July 24, 2024 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants